Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(CAT-1618) - Add code coverage back to ci #182

Merged
merged 1 commit into from
Jan 15, 2024
Merged

Conversation

jordanbreen28
Copy link

Summary

Provide a detailed description of all the changes present in this pull request.

Additional Context

Add any additional context about the problem here.

  • Root cause and the steps to reproduce. (If applicable)
  • Thought process behind the implementation.

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified.

@jordanbreen28 jordanbreen28 force-pushed the cat-1618-add_code_coverage branch 6 times, most recently from e5c2ec6 to 33c5258 Compare January 10, 2024 12:11
@jordanbreen28 jordanbreen28 reopened this Jan 10, 2024
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

@jordanbreen28 jordanbreen28 force-pushed the cat-1618-add_code_coverage branch from 33c5258 to a1795c9 Compare January 15, 2024 10:30
@jordanbreen28 jordanbreen28 marked this pull request as ready for review January 15, 2024 12:18
@jordanbreen28 jordanbreen28 requested review from bastelfreak and a team as code owners January 15, 2024 12:18
@jordanbreen28 jordanbreen28 force-pushed the cat-1618-add_code_coverage branch 3 times, most recently from 0d4dc21 to c5bb775 Compare January 15, 2024 12:21
@jordanbreen28 jordanbreen28 force-pushed the cat-1618-add_code_coverage branch from c5bb775 to 89ae51d Compare January 15, 2024 12:23
Copy link

@LukasAud LukasAud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LukasAud LukasAud merged commit 13b4025 into main Jan 15, 2024
7 checks passed
@LukasAud LukasAud deleted the cat-1618-add_code_coverage branch January 15, 2024 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants